Skip to content

fix: update cursor rule to say semgrep_scan#272

Merged
liukatkat merged 1 commit intodevelopfrom
katrina/cursor-rule-fix
Sep 24, 2025
Merged

fix: update cursor rule to say semgrep_scan#272
liukatkat merged 1 commit intodevelopfrom
katrina/cursor-rule-fix

Conversation

@liukatkat
Copy link
Contributor

@liukatkat liukatkat commented Sep 24, 2025

The cursor rule added by the MCP setup prompt has been updated to call semgrep_scan instead of security_check. We have also removed the security_check tool. We should update the cursor rule here to match.

Test plan: the cursor rule added should say semgrep_scan instead.

PR checklist:

  • Purpose of the code is evident to future readers
  • Tests included or PR comment includes a reproducible test plan
  • Documentation is up-to-date
  • A changelog entry was for any user-facing change
  • Change has no security implications (otherwise, ping security team)

If you're unsure about any of this, please see:

@liukatkat liukatkat changed the title security check -> semgrep_scan fix: update cursor rule to say semgrep_scan Sep 24, 2025
Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@liukatkat liukatkat marked this pull request as ready for review September 24, 2025 22:06
Copy link
Contributor Author

liukatkat commented Sep 24, 2025

Merge activity

  • Sep 24, 10:32 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Sep 24, 10:33 PM UTC: @liukatkat merged this pull request with Graphite.

@liukatkat liukatkat merged commit 3f5d5ed into develop Sep 24, 2025
12 checks passed
@liukatkat liukatkat deleted the katrina/cursor-rule-fix branch September 24, 2025 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants